-
Notifications
You must be signed in to change notification settings - Fork 1.4k
doc: install_ncs: nRFVSC on Open VSX Registry #24018
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:more detailsGithub labels
List of changed files detected by CI (0)
Outputs:ToolchainVersion: Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
be448af
to
c2721c4
Compare
doc/nrf/links.txt
Outdated
@@ -1692,6 +1692,8 @@ | |||
|
|||
.. ### Source: other (2 links or less from the same URL) | |||
|
|||
.. _`Open VSIX Registry`: https://open-vsx.org/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The website itself spells it: Open VSX Registry
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right -.- Thanks
doc/nrf/installation/install_ncs.rst
Outdated
.. note:: | ||
You can also use a different IDE compatible with the VSIX format and install the extensions that are part of the nRF Connect for VS Code Extension Pack from the `Open VSIX Registry`_. | ||
However, the extensions are only recommended and tested for use with |VSC|. | ||
Extension issues discovered in IDEs other than |VSC| will not be prioritized. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use same wording as before ... not reproducible on VSC IDE....
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While you are encouraged to report any issues you encounter on DevZone
_, issues discovered in editors other than |VSC| and not reproducible in |VSC| will not be prioritized.
c2721c4
to
41235d4
Compare
doc/nrf/test_and_optimize.rst
Outdated
@@ -71,7 +71,7 @@ Use one of the following methods: | |||
|
|||
.. group-tab:: nRF Connect for Visual Studio Code |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In other group-tabs you have nRF Connect for VS Code. Maybe just for consistency, have it like that also here.
Also wondering whether the shortcut would work in group-tabs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't look into shortcut, as it is not really something for this PR.
41235d4
to
2a44954
Compare
Updated documentation with information about nRFVSC being available on the Open VSX Registry website. Edited how nRFVSC is referred to in the docs. VSC-3067. Signed-off-by: Grzegorz Ferenc <[email protected]>
2a44954
to
bec28e2
Compare
@@ -33,7 +33,7 @@ For example, if you build for nRF52840 DK on the command line, you can use the f | |||
west build -b nrf52840dk/nrf52840 -- -DSHIELD=nrf21540ek | |||
If you use the |nRFVSC|, specify ``-DSHIELD=nrf21540ek`` in the **Extra Cmake arguments** field when `setting up a build configuration <How to work with build configurations_>`_. | |||
If you use |nRFVSC|, specify ``-DSHIELD=nrf21540ek`` in the **Extra Cmake arguments** field when `setting up a build configuration <How to work with build configurations_>`_. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you fix the typo in line 69 : In this command, the ipc_radio
image us used as default and builds the network core image with support for the combination of 802.15.4 and Bluetooth.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done in a separate commit.
Fixed typos and style in 21540ek_dev_guide.rst. NCSDK-NONE. Requested by @umapraseeda. Signed-off-by: Grzegorz Ferenc <[email protected]>
Updated documentation with information about nRFVSC being available on the Open VSIX Registry website. Edited how nRFVSC is referred to in the docs. VSC-3067.